font chooser: Don't set hardcoded default features
authorMatthias Clasen <mclasen@redhat.com>
Wed, 3 Jan 2018 18:41:20 +0000 (13:41 -0500)
committerMatthias Clasen <mclasen@redhat.com>
Wed, 3 Jan 2018 18:41:20 +0000 (13:41 -0500)
We should rely on harfbuzz to select the right defaults.

gtk/gtkfontchooserwidget.c

index c11fc9f9f036f7c240e924b463d977359150c2e0..ba7c19d66197cc0610e3386c739eb6d81ccf2686 100644 (file)
@@ -2087,7 +2087,7 @@ update_font_features (GtkFontChooserWidget *fontchooser)
   GString *s;
   GList *l;
 
-  s = g_string_new ("\"kern\" 1, \"curs\" 1, \"lfbd\" 1, \"rfbd\" 1, \"mark\" 1, \"mkmk\" 1, \"mset\" 1, \"ccmp\" 1, \"rlig\" 1, \"rclt\" 1, \"rvrn\" 1");
+  s = g_string_new ("");
 
   for (l = priv->feature_items; l; l = l->next)
     {
@@ -2101,7 +2101,9 @@ update_font_features (GtkFontChooserWidget *fontchooser)
           if (gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (item->feat)) &&
               strcmp (item->name, "xxxx") != 0)
             {
-              g_string_append (s, ", \"");
+              if (s->len > 0)
+                g_string_append (s, ", ");
+              g_string_append (s, "\"");
               g_string_append (s, item->name);
               g_string_append (s, "\" 1");
             }
@@ -2111,6 +2113,8 @@ update_font_features (GtkFontChooserWidget *fontchooser)
           if (gtk_check_button_get_inconsistent (GTK_CHECK_BUTTON (item->feat)))
             continue;
 
+          if (s->len > 0)
+            g_string_append (s, ", ");
           g_string_append (s, ", \"");
           g_string_append (s, item->name);
           if (gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (item->feat)))